Skip to content

AB#31228 - Added today date to email template.#2474

Merged
JamesPasta merged 2 commits into
devfrom
feature/AB#31228-Add-Today-Date-Variable
May 15, 2026
Merged

AB#31228 - Added today date to email template.#2474
JamesPasta merged 2 commits into
devfrom
feature/AB#31228-Add-Today-Date-Variable

Conversation

@hasanpour
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds support for a “today’s date” token in notification email templates, enabling templates to include a current date value and ensuring previously-saved drafts refresh that date when reopened in the email editor.

Changes:

  • Added a today_date template variable to notifications data seeding.
  • Implemented client-side rendering of today_date via Handlebars and inserted a marker span (data-token="today_date") into the rendered HTML.
  • Added logic to refresh any existing today_date spans to the current date when loading an email body into TinyMCE / the backing form field.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
applications/Unity.GrantManager/src/Unity.GrantManager.Web/Views/Shared/Components/EmailsWidget/Default.js Adds generation + refresh logic for a today_date token in rendered email HTML.
applications/Unity.GrantManager/modules/Unity.Notifications/src/Unity.Notifications.Domain/NotificationsDataSeedContributor.cs Seeds a new today_date template variable so it’s available for templates.

@github-actions
Copy link
Copy Markdown

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 734
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 734
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

@JamesPasta JamesPasta merged commit db34d3e into dev May 15, 2026
25 checks passed
@hasanpour hasanpour deleted the feature/AB#31228-Add-Today-Date-Variable branch May 15, 2026 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants